Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

smart-phone-analysis #697

Merged
merged 1 commit into from
Oct 31, 2024
Merged

smart-phone-analysis #697

merged 1 commit into from
Oct 31, 2024

Conversation

BenakDeepak
Copy link
Contributor

Here's a concise update based on the format you've specified:

Related Issues or Bug

  • Lack of additional predictive features may limit model accuracy.
  • Dataset might need further cleaning to reflect realistic data distributions.

Fixes: #688

Proposed Changes

  • Added columns like brand, model, and storage capacity as predictors for improved accuracy.
  • Filled NaN values in the Rating column and adjusted the distribution to ensure realistic values.
  • Optimized RandomForestRegressor with GridSearchCV for hyperparameter tuning.
  • Compared RandomForestRegressor with GradientBoostingRegressor and XGBoost for model selection.
  • Created log transformation on Reviews due to high range and added interaction terms between Rating and Reviews.

Additional Info

  • Conducted exploratory data analysis (EDA) to understand data patterns and significant predictors.
  • Added Mean Absolute Error (MAE) as an evaluation metric alongside RMSE for more comprehensive error assessment.

Screenshots

Original Updated
Original screenshot Updated screenshot

@github-actions GitHub Actions
Copy link

👋 Thank you for opening this pull request! We appreciate your contribution to improving this project. Your PR is under review, and we'll get back to you shortly.
Don't forget to mention the issue you solved!.

To help move the process along, please tag @UppuluriKalyani, @Neilblaze, and @SaiNivedh26 for a faster review!

@UppuluriKalyani UppuluriKalyani merged commit 574e2e6 into UppuluriKalyani:main Oct 31, 2024
3 checks passed
@github-actions GitHub Actions
Copy link

🎉🎉 Thank you for your contribution! Your PR #697 has been merged! 🎉🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@UppuluriKalyani UppuluriKalyani UppuluriKalyani approved these changes

@Neilblaze Neilblaze Awaiting requested review from Neilblaze

@SaiNivedh26 SaiNivedh26 Awaiting requested review from SaiNivedh26

Assignees

@BenakDeepak BenakDeepak

Labels
gssoc-ext hacktoberfest-accepted level1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature request Smart-Phone prediction
2 participants
@BenakDeepak @UppuluriKalyani